Skip to content

Mcp authorization control#728

Merged
canptura merged 11 commits intomainfrom
mcp-authorization-control
May 4, 2026
Merged

Mcp authorization control#728
canptura merged 11 commits intomainfrom
mcp-authorization-control

Conversation

@jjoderis
Copy link
Copy Markdown
Contributor

Summary

Added authorization checking to the mcp tools to prevent users from accessing functionality or data they would not have access to through the web interface.

Details

  • Fixed: The role permission interface for instance management actually showed and changed the permission for instance viewing
  • Removed the example prompts and resources since they are not used and are also not really doing anything useful yet
  • added permission checking to all available mcp tools to ensure that users cannot do things they would not be able to do through the web interface
    • added new tool that lists the tools that can actually be used by the user given the space settings and the users permissions
      • xmcp currently does not seem to allow toggling which tools are advertised at run time so we cannot deactivate tools based on env variables
    • if an LLM tries to access a tool that the user cannot use it will get an error message

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Comment thread src/management-system-v2/lib/mcp-utils.ts
@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

CLOUDRUN ACTIONS

✅ Successfully created Preview Deployment.

https://pr-728---ms-server-staging-c4f6qdpj7q-ew.a.run.app

Copy link
Copy Markdown
Contributor

@canptura canptura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@canptura canptura merged commit 8ca3a39 into main May 4, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants